-
Couldn't load subscription status.
- Fork 2k
Update README instructions for cloning and building #4679
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
Git Large File Storage is required to clone the project. To build the project a container runtime (i.e. Docker Desktop) is required to be running locally. Signed-off-by: Robert McNees <86265089+robertmcnees@users.noreply.github.com>
|
|
||
| - Ignore the large files (won't affect the spring-ai behaviour) : `GIT_LFS_SKIP_SMUDGE=1 git clone git@github.com:spring-projects/spring-ai.git`. | ||
| - Or install the [Git Large File Storage](https://git-lfs.com/) before cloning the repo. | ||
| - Install the [Git Large File Storage](https://git-lfs.com/) before cloning the repo. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I frankly don't even know this is needed. I never had an issue cloning (on ubuntu) and I others on the team on Mac don't have this large file support installed. Can you tell us what happens? I'd like to not have this dependency to clone of course ,but there seems to be a deeper issue here.
Thanks!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe it's local to my machine. I pasted my experience below in detail. It's a small change so if it's not true for everyone feel free to close the PR.
|
Additional context on my experience cloning the repository. I was using https, not ssh. No git-lfs No git-lfs In both cases the folder only checked out a subset of files: In both instances, this error message seems relevant: Could that be because of .gitattributes? Once I installed With git-lfs |

Minor changes to the README to include two small tasks to get the code to clone and compile locally.
git-lfsis requiredGit Large File Storage is required to clone the project. Even with the command
GIT_LFS_SKIP_SMUDGE=1I still got the error:After a
brew install git-lfsthings worked fine. I verified after that theGIT_LFS_SKIP_SMUDGE=1did reduce the disk size of the download. Butgit-lfsseems to be a requirement.Run Docker Desktop
To build the project I needed a container runtime (i.e. Docker Desktop) running locally.